Skip to content

Conversation

@sarthakdeva-intel
Copy link
Contributor

Description

This PR adds the initial code for smart intersection agent.

Fixes # (issue)

Any Newly Introduced Dependencies

Please describe any newly introduced 3rd party dependencies in this change. List their name, license information and how they are used in the project.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Checklist:

  • I agree to use the APACHE-2.0 license for my code changes.
  • I have not introduced any 3rd party components incompatible with APACHE-2.0.
  • I have not included any company confidential information, trade secret, password or security token.
  • I have performed a self-review of my code.

@krish918
Copy link
Contributor

krish918 commented Jan 2, 2026

Does it also contain the updates for removing gamification, or is it just bringing the previous code to the repo?

@sarthakdeva-intel
Copy link
Contributor Author

Does it also contain the updates for removing gamification, or is it just bringing the previous code to the repo?

yes, removed gamification. Retaining default weather. This follows older way of deployment.

@krish918
Copy link
Contributor

krish918 commented Jan 6, 2026

Does it also contain the updates for removing gamification, or is it just bringing the previous code to the repo?

yes, removed gamification. Retaining default weather. This follows older way of deployment.

@sarthakdeva-intel Is it possible to have one separate commit capturing only the removal of the gamification part?

Signed-off-by: Deva, Sarthak <sarthak.deva@intel.com>
Signed-off-by: Deva, Sarthak <sarthak.deva@intel.com>
Signed-off-by: Deva, Sarthak <sarthak.deva@intel.com>
Signed-off-by: Krishna Murti <krishna.murti@intel.com>
- Used pip to install pinned version of uv
- Updated uv command to install dependencies using pyproject and lock file
- Removed un-necessary lines

Signed-off-by: Krishna Murti <krishna.murti@intel.com>
madhuri-rai07
madhuri-rai07 previously approved these changes Jan 8, 2026
Copy link
Contributor

@madhuri-rai07 madhuri-rai07 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All remaining comments to be resolved in separate PR, hence approving

bharagha
bharagha previously approved these changes Jan 9, 2026
@bharagha
Copy link
Contributor

bharagha commented Jan 9, 2026

The conversations must be resolved for the merge to happen. Request to do that post creating necessary tickets and proceed with merge.

- Putting source code in src/ instead of src/traffic-intelligence for uniformity with other applications

Signed-off-by: Krishna Murti <krishna.murti@intel.com>
- Moved gitignore one level up to the root of application
- Added edge-ai-suites and some other files/dir to gitignore
- Added venv and some other files/dirs to dockerignore
- Updated version for vlm-openvino-serving image to 1.3.1
- Removed explicit container names as they cause conflicts in dev envs

Signed-off-by: Krishna Murti <krishna.murti@intel.com>
@krish918 krish918 dismissed stale reviews from madhuri-rai07 and bharagha via 63030e3 January 9, 2026 07:46
@madhuri-rai07 madhuri-rai07 merged commit 92633a0 into open-edge-platform:main Jan 9, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants